Skip to content

Comments

feat: add telemetry for cli#1205

Open
radugheo wants to merge 1 commit intomainfrom
feat/add-telemetry
Open

feat: add telemetry for cli#1205
radugheo wants to merge 1 commit intomainfrom
feat/add-telemetry

Conversation

@radugheo
Copy link
Collaborator

@radugheo radugheo commented Jan 26, 2026 •
edited by github-actions bot
Loading

Added CLI telemetry tracking via Application Insights for all commands, reporting execution status, duration, and environment context.
Events are buffered in-process and flushed to App Insights via an atexit handler so telemetry never blocks command execution.

Development Package

  • Use uipath pack --nolock to get the latest dev build from this PR (requires version range).
  • Add this package as a dependency in your pyproject.toml:
[project]
dependencies = [
  # Exact version:
  "uipath==2.9.2.dev1012055002",

  # Any version from PR
  "uipath>=2.9.2.dev1012050000,<2.9.2.dev1012060000"
]

[[tool.uv.index]]
name = "testpypi"
url = "https://test.pypi.org/simple/"
publish-url = "https://test.pypi.org/legacy/"
explicit = true

[tool.uv.sources]
uipath = { index = "testpypi" }

[tool.uv]
override-dependencies = [
    "uipath>=2.9.2.dev1012050000,<2.9.2.dev1012060000",
]

@github-actions github-actions bot added test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-llamaindex Triggers tests in the uipath-llamaindex-python repository labels Jan 26, 2026
@radugheo radugheo force-pushed the feat/add-telemetry branch 3 times, most recently from 2708d1e to cab218d Compare January 26, 2026 17:41
@radugheo radugheo added the build:dev Create a dev build from the pr label Jan 30, 2026
@radugheo radugheo force-pushed the feat/add-telemetry branch 3 times, most recently from 2235c65 to 0a28051 Compare January 30, 2026 14:50
@radugheo radugheo force-pushed the feat/add-telemetry branch 17 times, most recently from b6c241e to 146a9f7 Compare February 12, 2026 16:51
@radugheo radugheo removed the build:dev Create a dev build from the pr label Feb 12, 2026
@radugheo radugheo force-pushed the feat/add-telemetry branch 2 times, most recently from b3272a8 to 091be5f Compare February 12, 2026 17:00
@radugheo radugheo changed the title [WIP] feat: add telemetry for cli feat: add telemetry for cli Feb 12, 2026
@radugheo radugheo marked this pull request as ready for review February 13, 2026 08:02
radu-mocanu
radu-mocanu reviewed Feb 18, 2026
View reviewed changes
radu-mocanu
radu-mocanu reviewed Feb 18, 2026
View reviewed changes
@radugheo radugheo force-pushed the feat/add-telemetry branch 3 times, most recently from 6a0e0fb to 642aa3e Compare February 20, 2026 15:24
@radugheo radugheo added the build:dev Create a dev build from the pr label Feb 20, 2026
@radugheo radugheo force-pushed the feat/add-telemetry branch 3 times, most recently from c580935 to 6da430b Compare February 23, 2026 16:21
radu-mocanu
radu-mocanu reviewed Feb 23, 2026
View reviewed changes
src/uipath/telemetry/_track.py Outdated
_logger.propagate = False


def _get_connection_string() -> Optional[str]:
Copy link
Collaborator

@radu-mocanu radu-mocanu Feb 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit

Suggested change
def _get_connection_string() -> Optional[str]:
def _get_connection_string() -> str | None:

radu-mocanu
radu-mocanu reviewed Feb 23, 2026
View reviewed changes
src/uipath/_cli/_telemetry.py Outdated
properties["CloudOrganizationId"] = UiPathConfig.organization_id

# Get tenant ID
tenant_id = os.getenv("UIPATH_TENANT_ID")
Copy link
Collaborator

@radu-mocanu radu-mocanu Feb 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can add this to UiPathConfig as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@radu-mocanu radu-mocanu radu-mocanu left review comments

@cristipufu cristipufu Awaiting requested review from cristipufu

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

build:dev Create a dev build from the pr test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-llamaindex Triggers tests in the uipath-llamaindex-python repository

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@radugheo @radu-mocanu